-
Notifications
You must be signed in to change notification settings - Fork 1.8k
Add new lint shadow_type_generic
#15109
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Add new lint shadow_type_generic
#15109
Conversation
69325c4 to
cf09da7
Compare
|
r? clippy |
|
Dear contributor. Clippy has entered a feature freeze period and will not be integrating new lints for a few weeks. You can rest assured that the Clippy team will review and discuss your proposal at the end of the feature freeze period. If you wish to contribute to Clippy in the meantime, do not hesitate to read the pinned tracking issue about the feature freeze itself. @rustbot label +A-lint +S-blocked -S-waiting-on-review |
This comment has been minimized.
This comment has been minimized.
cf09da7 to
b52802f
Compare
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Add a new restriction lint `shadow_type_generic` that checks for generic parameters that shadow types in scope. Signed-off-by: Alexandre Barone <[email protected]>
b52802f to
480ae1c
Compare
|
This PR was rebased onto a different master commit. Here's a range-diff highlighting what actually changed. Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers. |
|
r? clippy I just saw that this PR is active again. Re-assigning, as I don't have review capacity at the moment. (I wonder why |
Add a new restriction lint
shadow_type_genericthat checks for generic parameters that shadow types in scope.Closes: #380
changelog: [
shadow_type_generic]: new lintSummary Notes
Managed by
@rustbot—see help for details